-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: setup jest
in Stimulus variants
#465
Conversation
9d9fca2
to
f69989f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
can we have this also with the installed github actions ci checks? i think bin/ci-run is only for testing when in this repo
Yup that's tracked by #422 but I want to do it in another PR |
f6e00c1
to
1655045
Compare
1655045
to
93031a4
Compare
tbh I feel like this should be pulled into either its own variants or into the |
This should make it easier to write tests directly for Stimulus controllers (and JavaScript in general) which can be nice as they're faster than the current alternative (system tests).